Skip to content

Conversation

@kdeakinstructure
Copy link
Contributor

@kdeakinstructure kdeakinstructure commented Oct 10, 2025

Fixed a bug where on the Syllabus Summary page the quizzes icons was not shown correctly. (All classic quizzes had locked icon, and all new quizzes had the default assignment icon).

Test Plan: Check all the 3 apps if the quizzes (whether classic or new) has the quiz icon. (In Teacher we intentionally don't show locked icons).

refs: MBL-19286
affects: Student, Teacher, Parent
release note: Fix a bug where Syllabus Summary pages does not show correct icons for classic (locked) and new quizzes (default assignment icon).

  • Tested in dark mode

…not shown correctly. (All classic quizzes had locked icon, and all new quizzes had the default assignment icon).

refs: MBL-18749
affects: Student, Teacher, Parent
release note: Fix a bug where Syllabus Summary pages does not show correct icons for classic (locked) and new quizzes (default assignment icon).
@inst-danger
Copy link
Contributor

Parent Install Page

@github-actions
Copy link

📊 Code Coverage Report

⚠️ Student

  • PR Coverage: 43.35%
  • Master Coverage: 43.38%
  • Delta: -0.02%

⚠️ Teacher

  • PR Coverage: 24.40%
  • Master Coverage: 24.42%
  • Delta: -0.01%

✅ Pandautils

  • PR Coverage: 22.18%
  • Master Coverage: 22.18%
  • Delta: +0.00%

📈 Overall Average

  • PR Coverage: 29.98%
  • Master Coverage: 29.99%
  • Delta: -0.01%

@inst-danger
Copy link
Contributor

Teacher Install Page

@inst-danger
Copy link
Contributor

Student Install Page

Copy link
Contributor

@adamNagy56 adamNagy56 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QA +1

@tamaskozmer tamaskozmer self-requested a review October 14, 2025 09:35
@kdeakinstructure kdeakinstructure merged commit 2189545 into master Oct 14, 2025
6 of 7 checks passed
@kdeakinstructure kdeakinstructure deleted the MBL-19286-wrong-icons-on-parent-summary-quizzes branch October 14, 2025 21:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants